Skip to content

feat!: CLAUDE-GENERATED deduplication using contract address for bc hash, bc decomp, instr fetch#15964

Closed
dbanks12 wants to merge 14 commits intonextfrom
db/dedup-contract-address
Closed

feat!: CLAUDE-GENERATED deduplication using contract address for bc hash, bc decomp, instr fetch#15964
dbanks12 wants to merge 14 commits intonextfrom
db/dedup-contract-address

Conversation

@dbanks12
Copy link
Contributor

Please read contributing guidelines and remove this line.

For audit-related pull requests, please use the audit PR template.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Comment on lines 31 to 38
vinfo("Contract ", field_to_string(address), " is not deployed!");
throw BytecodeNotFoundError(bytecode_id, "Contract " + field_to_string(address) + " is not deployed");
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wunderaligned-exception-object"
throw BytecodeNotFoundError(address, "Contract " + field_to_string(address) + " is not deployed");
#pragma clang diagnostic pop
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol nice one claude

Base automatically changed from merge-train/avm to next July 25, 2025 18:51
@dbanks12 dbanks12 changed the title feat!: CLAUDE-GENERATED deduplication using class ID for some things, but contract address for instr fetching. feat!: CLAUDE-GENERATED deduplication using contract address for most things Jul 28, 2025
@dbanks12 dbanks12 changed the title feat!: CLAUDE-GENERATED deduplication using contract address for most things feat!: CLAUDE-GENERATED deduplication using contract address for bc hash, bc decomp, instr fetch Jul 28, 2025
@dbanks12
Copy link
Contributor Author

dbanks12 commented Aug 5, 2025

Replaced with #15977

@dbanks12 dbanks12 closed this Aug 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant